Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Oct 8, 2025

What do these changes do?

This PR fixes an issue with the "Escalate to Support" functionality in Sim4Life

In the demo, since I can't start S4imLife locally, I use a JLab that mimics the "Escalate to Support" behaviour:
EscalateToSupport

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Oct 8, 2025
@odeimaiz odeimaiz added this to the Cheops milestone Oct 8, 2025
@odeimaiz odeimaiz added bug buggy, it does not work as expected a:frontend issue affecting the front-end (area group) labels Oct 8, 2025
@mergify
Copy link
Contributor

mergify bot commented Oct 8, 2025

🧪 CI Insights

Here's what we observed from your CI run for 885d552.

🟢 All jobs passed!

But CI Insights is watching 👀

@odeimaiz odeimaiz requested a review from Copilot October 8, 2025 11:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes an issue with the "Escalate to Support" functionality in Sim4Life by updating a method call in the PersistentIframe component. The change corrects the method name used when creating a support conversation with escalated content.

  • Updated method call from proposeConversation to createConversation for escalating support requests

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@odeimaiz odeimaiz requested review from alexpargon and mguidon October 8, 2025 11:56
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 8, 2025

@odeimaiz odeimaiz enabled auto-merge (squash) October 8, 2025 12:10
@odeimaiz odeimaiz merged commit ef6f3f1 into ITISFoundation:master Oct 8, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants